Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: implement a method to delete a directory from a template. #1199

Open
wants to merge 3 commits into
base: nightly
Choose a base branch
from

Conversation

GiantTreeLP
Copy link
Member

Motivation

See #1198.

Modification

This change allows developers to directly delete a directory and all its containing files and directories from a given service template.

As an additional functionality, this method is also exposed to the HTTP Rest API much like the file deletion method already is.

Result

Developers can now delete entire directory trees from a given template.

Other context

Closes #1198.

Local template storage has already been tested on my machine, I will check whether SFTP and S3 storage can be tested. If not, I may seek help from your end.

@GiantTreeLP GiantTreeLP force-pushed the 1198-delete-directories-in-templates branch 2 times, most recently from a6da5fe to 72328b0 Compare April 15, 2023 22:58
@GiantTreeLP GiantTreeLP force-pushed the 1198-delete-directories-in-templates branch from 72328b0 to 785e4a6 Compare April 30, 2023 20:25
@GiantTreeLP GiantTreeLP reopened this Apr 30, 2023
@GiantTreeLP GiantTreeLP force-pushed the 1198-delete-directories-in-templates branch from 56b9a3f to 83b3b96 Compare May 8, 2023 12:18
@GiantTreeLP GiantTreeLP force-pushed the 1198-delete-directories-in-templates branch from 83b3b96 to efd9cbd Compare July 9, 2023 15:13
@GiantTreeLP GiantTreeLP marked this pull request as ready for review July 9, 2023 15:34
@0utplay 0utplay added this to the 4.0.0-RC10 milestone Jul 12, 2023
@0utplay 0utplay requested a review from derklaro July 12, 2023 14:30
GiantTreeLP and others added 3 commits October 2, 2023 23:50
This change allows developers to directly delete a directory and all its containing files and directories from a given service template.

As an additional functionality, this method is also exposed to the HTTP Rest API much like the file deletion method already is.
@GiantTreeLP GiantTreeLP force-pushed the 1198-delete-directories-in-templates branch from c9bd3e6 to 154ac82 Compare October 2, 2023 21:50
Copy link
Contributor

@MarkusTieger MarkusTieger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github said "At least 1 approving review is required to merge this pull request.", so i made one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a method to delete entire directory trees from templates
3 participants